Skip to content

feat: Add restartCount as container golden metric#3072

Open
hannahsohkim wants to merge 4 commits into
newrelic:mainfrom
hannahsohkim:hkim/NR-590898
Open

feat: Add restartCount as container golden metric#3072
hannahsohkim wants to merge 4 commits into
newrelic:mainfrom
hannahsohkim:hkim/NR-590898

Conversation

@hannahsohkim

@hannahsohkim hannahsohkim commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Relevant information

Added a new golden metric restartCount to the container definition. Adding in staging first to test, then will open a new PR to add to the definition.

Api Review Board (ARB)

Any pull request with changes to this repository has to be linked with an ARB ticket, which has to be approved before merging.

If you are an external contributor please contact New Relic Support.

If you don't know about the ARB process check this

ARB Jira ticket:
https://new-relic.atlassian.net/browse/NR-XXXX

Checklist

  • I've read the guidelines and understand the acceptance criteria.
  • The value of the attribute marked as identifier will be unique and valid.
  • I've confirmed that my entity type wasn't already defined. If it is I'm providing an explanation above.
  • I've linked an ARB ticket & received approval from API Review Board in order to make these changes

@entityBot entityBot left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

beep boop bop.

I have found some errors processing these changes:

  • Error VALIDATING domainType: 'INFRA-CONTAINER' on Summary metrics for key ''containerRestart': wrong golden metric reference. Couldn't find a golden metric definition with key 'containerRestart', please populate 'goldenMetric' with a valid reference to an existing golden metric.' - Exception: 'containerRestart': wrong golden metric reference. Couldn't find a golden metric definition with key 'containerRestart', please populate 'goldenMetric' with a valid reference to an existing golden metric.
  • No valid, approved ARB ticket found in the PR description. This PR has to be reviewed by the API Review Board, please include a link to the ticket of that review in the PR description. Check how to start the process here or reach #api-review-board channel in case of doubts

@entityBot
entityBot requested review from a team July 21, 2026 22:33

@entityBot entityBot left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We don't think these changes require an API Review Board (ARB) review, so it is not required to merge this PR. Note that all API changes do require one: if you believe this PR needs an ARB review, please create the ticket and add the ARB:pending label. Reach #api-review-board in case of doubts.

@entityBot
entityBot requested review from a team and removed request for a team July 21, 2026 22:55

@entityBot entityBot left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We don't think these changes require an API Review Board (ARB) review, so it is not required to merge this PR. Note that all API changes do require one: if you believe this PR needs an ARB review, please create the ticket and add the ARB:pending label. Reach #api-review-board in case of doubts.

@entityBot entityBot left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We don't think these changes require an API Review Board (ARB) review, so it is not required to merge this PR. Note that all API changes do require one: if you believe this PR needs an ARB review, please create the ticket and add the ARB:pending label. Reach #api-review-board in case of doubts.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants